tests/pkg/lwip: include default-radio-settings.inc.mk [backport 2024.01] #20330
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Backport of #20325
Contribution description
For the release tests we decided to randomize the PAN ID to not interfere with ongoing RIOT-based experiments in the testbed (see RIOT-OS/Release-Specs#300). However, lwIP currently does not include the Makefile to set the PAN ID at compile time. This fixes that.
Testing procedure
Apply the following patch
Then compile and flash
tests/pkg/lwip
to a board that has an IEEE 802.15.4 radio. Open a terminal and reboot. The node should now printPANID: 0x0023
. If you then recompile and flashtests/pkg/lwip
withDEFAULT_PAN_ID
set to another value in the environment variables, e.g.DEFAULT_PAN_ID=0x1234
, without this PR, the output will still bePANID: 0x0023
, with this PR it will bePANID: 0x1234
.Issues/PRs references
Required for RIOT-OS/Release-Specs#300.